Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pr 리뷰 알림 발동 조건 변경 #82

Merged
merged 1 commit into from
Nov 12, 2024
Merged

pr 리뷰 알림 발동 조건 변경 #82

merged 1 commit into from
Nov 12, 2024

Conversation

parkblo
Copy link
Collaborator

@parkblo parkblo commented Nov 7, 2024

✏️ 한 줄 설명

이 PR의 주요 변경 사항이나 구현된 내용을 간략히 설명해 주세요.

  • codeowners를 통한 리뷰어 자동 등록이 review request를 3번 발동시키는 현상때문에 알람이 여러번 전송되는 문제 해결을 위함

✅ 작업 내용

  • github action 발동 조건 변경

🏷️ 관련 이슈

📌 리뷰 진행 시 참고 사항

리뷰 코멘트 작성 시 특정 사실에 대해 짚는 것이 아니라 코드에 대한 의견을 제안할 경우, 강도를 함께 제시해주세요! (1점: 가볍게 참고해봐도 좋을듯 ↔ 5점: 꼭 바꾸는 게 좋을 것 같음!)

예상치 못하게 슬랙봇을 잘 동작시키기가 어렵네요. ㅎㅎ 커밋 이력을 더 더럽히지 않기 위해서 앞으로는 main 브랜치 수정이 아닌 PR을 꼭 올리도록 하겠습니다.

- codeowners를 통한 리뷰어 자동 등록이 review request를 3번 발동시키는 현상때문에 알람이 여러번 전송되는 문제 해결을 위함
- web29 팀에 대한 리뷰 요청이 마지막에 취소되는 것을 이용하여 발동 조건을 `review_request_remove`로 설정하였음.
@parkblo parkblo self-assigned this Nov 7, 2024
@parkblo parkblo requested a review from a team as a code owner November 7, 2024 08:16
@parkblo parkblo requested review from heegenie and fru1tworld and removed request for a team November 7, 2024 08:16
@parkblo parkblo linked an issue Nov 7, 2024 that may be closed by this pull request
3 tasks
Copy link
Collaborator

@heegenie heegenie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

따봉비봇아 고마워 🐝

@fru1tworld
Copy link
Collaborator

따봉비봇아 고마워 🐝

@parkblo parkblo merged commit 0a6a39e into dev-fe Nov 12, 2024
1 check passed
heegenie added a commit that referenced this pull request Nov 12, 2024
* fix: pr 리뷰 알림 발동 조건 변경 (#82)

chore: pr 리뷰 알림 발동 조건 변경

- codeowners를 통한 리뷰어 자동 등록이 review request를 3번 발동시키는 현상때문에 알람이 여러번 전송되는 문제 해결을 위함
- web29 팀에 대한 리뷰 요청이 마지막에 취소되는 것을 이용하여 발동 조건을 `review_request_remove`로 설정하였음.

* feat: 기본 node 컴포넌트 구현

* feat: head node UI 구현

* feat: 노트 노드 ui 구현

---------

Co-authored-by: Byeongju Park <[email protected]>
Co-authored-by: CatyJazzy <[email protected]>
Co-authored-by: heegenie <[email protected]>
Co-authored-by: Heejin Na <[email protected]>
@parkblo parkblo deleted the fix/beebot branch November 12, 2024 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PR 리뷰 요청 알림 메시지가 여러번 전송되는 문제
3 participants